Skip to content

Add files via upload#7

Open
GAS53 wants to merge 1 commit intomainfrom
lesson_6
Open

Add files via upload#7
GAS53 wants to merge 1 commit intomainfrom
lesson_6

Conversation

@GAS53
Copy link
Copy Markdown
Owner

@GAS53 GAS53 commented Apr 17, 2022

No description provided.

Copy link
Copy Markdown

@Akseonov Akseonov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

все хорошо, замечаний нет

Comment on lines +14 to +15
const title = colors[randomInt(0, colors.length - 1)] + ' ' + entities[randomInt(0, entities.length - 1)];
return title
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

переменная title избыточна, можно было сразу вернуть все то, что присваивается title

Comment on lines +85 to +86
let = productRow = `
<tr>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

после let знак равно лишний

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants